Skip to content

Fix syntax for superellipse function in syntaxes.json#1042

Merged
dipikabh merged 1 commit intomdn:mainfrom
chs2019:patch-1
Feb 6, 2026
Merged

Fix syntax for superellipse function in syntaxes.json#1042
dipikabh merged 1 commit intomdn:mainfrom
chs2019:patch-1

Conversation

@chs2019
Copy link
Contributor

@chs2019 chs2019 commented Jan 31, 2026

parameter options should be grouped to an array of terms

Description

I put the parameter options of superellipse() in brackets to group them into an array.

Motivation

The current version breaks parsers, for example CSSTree as used in csstype

Additional details

I followed the pattern as seen in skewX() in the same file.

Alternatively, a new "type" can be created and used like so:

  "superellipse-domain": {
    "syntax": "<number> | infinity | -infinity"
  },
  "superellipse()": {
    "syntax": "superellipse( <superellipse-domain> )"
  },

Related issues and pull requests

parameter options must be grouped to an array of terms
@chs2019 chs2019 requested a review from a team as a code owner January 31, 2026 11:40
@chs2019 chs2019 requested a review from dipikabh January 31, 2026 11:40
Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks a lot for sending this change.

@dipikabh dipikabh merged commit ce27567 into mdn:main Feb 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants